fix(joex): migrate office conversion defaults to unoserver - #3347
Conversation
Switch the default convert command from deprecated unoconv to unoconvert so joex works with a unoserver daemon that queues concurrent LibreOffice converts (eikek#3345). Co-authored-by: Cursor <cursoragent@cursor.com>
|
Tick the box to add this pull request to the merge queue (same as
|
|
Companion docker PR: docspell/docker#44 |
|
Put on hold due to discussion in docspell/docker#44. @eikek do you agree it would be better to host unoserver on the joex instance? |
|
Hi @tiborrr thank you for your efforts! I'm not very sure what is meant to "host unoserver on the joex instance". I checked docspell/docker#44 and also agree that it is currently not e nice solution (famous quick temporary thing :)). So my personal preference would be to have some basic command that is possible to "just call" as a simple default and have all that unoserver stuff (as a better conversion alternative) implemented out of band. Sometimes a sidecar container might do it or on linux a systemd service can be used to maintain unoserver…. Then the command can be adopted in the config accordingly. I'm also fine disabling this feature and document the steps to make it work. Of course, the docker repo can provide it out of the box. What do you think? |
2 participants
Summary
docspell.joex.convert.unoconvcommand from deprecatedunoconvtounoconvert(HOCON key kept for compatibility)unoserverdaemon; concurrent converts are queued by the serverBreaking
Custom / non-Docker installs that used
unoconv -lmust startunoserverinstead and provideunoconvertonPATH. Official Docker changes are in a companion PR on docspell/docker.Test plan
reference.confscheduler.pool-size > 1, multiple.xlsxuploads convert without wedging joexunoserverand waits forunopingRelated: #3345 #3293
Made with Cursor